-
Notifications
You must be signed in to change notification settings - Fork 987
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Onboarding remove identifiers code and refactor create_or_sync_profile screen #21668
Conversation
Jenkins BuildsClick to see older builds (55)
|
860e77b
to
336f1cb
Compare
336f1cb
to
cf765f2
Compare
dc4ad41
to
1c6e970
Compare
e414922
to
063a94c
Compare
@@ -0,0 +1,101 @@ | |||
(ns status-im.contexts.onboarding.create-profile.view |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Refactored the create-or-sync-profile screen and split it into two separate screens:
('Create Profile' and 'Log In - as per figma)
063a94c
to
85c0511
Compare
[utils.i18n :as i18n] | ||
[utils.re-frame :as rf])) | ||
|
||
(defn- navigate-to-create-profile-password |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don we have this in every control ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, @flexsurfer, for reviewing the PR.
Are you asking if we need the create-profile-password in both the create-profile and login flows? Yes, we do.
For the create profile flow, we are directly navigating to the create-profile-password screen here. However, for the login flow, navigation to the password screen occurs after the seed phrase is validated.
38% of end-end tests have passed
Failed tests (5)Click to expandClass TestWalletMultipleDevice:
Class TestWalletOneDevice:
Class TestCommunityMultipleDeviceMerged:
Passed tests (3)Click to expandClass TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
|
91% of end-end tests have passed
Failed tests (3)Click to expandClass TestWalletMultipleDevice:
Class TestCommunityMultipleDeviceMergedTwo:
Expected to fail tests (2)Click to expandClass TestCommunityMultipleDeviceMergedTwo:
Passed tests (51)Click to expandClass TestActivityMultipleDevicePR:
Class TestCommunityMultipleDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestActivityMultipleDevicePRTwo:
Class TestCommunityOneDeviceMerged:
Class TestWalletOneDevice:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestFallbackMultipleDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestDeepLinksOneDevice:
|
Hey @Parveshdhull ! Tested PR on both platforms. |
4eb38c7
to
79ffc74
Compare
Summary
create-or-sync-profile
screen and breaks into two different screens (create-profile and log-in)Testing
Please lightly test onboarding flow to make sure cleanup is not breaking it.
status: ready